-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support regexp in command builder on the project name #1419
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1419 +/- ##
==========================================
- Coverage 69.53% 69.52% -0.01%
==========================================
Files 93 93
Lines 6299 6327 +28
==========================================
+ Hits 4380 4399 +19
- Misses 1542 1550 +8
- Partials 377 378 +1
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Just realized one thing that becomes a bit confusing. Can you think of a good way to make this more restrictive? Maybe gating this feature with a server flag would be enough? |
Good point, I guess put this feature behind a server flag is a good idea. |
This feature is now disabled by default and managed by the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@bewie before I merge this, can you either update this one with docs on the new flag (and specifically call out that it's not meant to be used with disable-apply-all) or create a new PR for this? |
ffa3dad
to
bd80e52
Compare
@nishkrishnan I updated the docs, let me know if it looks good to you. |
Related to #1288, this PR adds support of regexp on atlantis plan/apply with
-p
flag.The main objective is to be able to trigger apply only on a specific range of projects in one GitHub comment.
For Example, all projects that matching my naming convention for staging :
Or a specific range of project :
This feature can be enabled with
--enable-regexp-cmd
flag (disable by default)Closes #254